@culka reviewed 13 files and all commit messages, and made 2 comments.
Reviewable status: all files reviewed, 2 unresolved discussions (waiting on @jannebe).

import_digiroad_shapefiles.sh line 26 at r1 (raw file):

    echo $DIGIROAD_IRROTUS_NRO > "${DOWNLOAD_TARGET_DIR}/digiroad_${DIGIROAD_IRROTUS_NRO}.txt"
  fi
  rm -f "${DOWNLOAD_TARGET_DIR}/digiroad_irrotus_nro.txt"

Sinänsä turha tehdä poisto, kun alla oleva rivi kuitenkin ylikirjoittaa mitä tiedostossa olisi ollut.

import_digiroad_shapefiles.sh line 27 at r1 (raw file):

  fi
  rm -f "${DOWNLOAD_TARGET_DIR}/digiroad_irrotus_nro.txt"
  echo $DIGIROAD_IRROTUS_NRO > "${DOWNLOAD_TARGET_DIR}/digiroad_irrotus_nro.txt"

Miksi sama numero tallennetaan sekä digiroad_${DIGIROAD_IRROTUS_NRO}.txt hen että digiroad_irrotus_nro.txt:hen?

Ylempää näistä ei taideta lukea missään?

@jannebe resolved 1 discussion.
Reviewable status: all files reviewed, 1 unresolved discussion (waiting on @culka).

@culka made 2 comments and resolved 1 discussion.
Reviewable status: complete! all files reviewed, all discussions resolved (waiting on @jannebe).

import_digiroad_shapefiles.sh line 26 at r1 (raw file):

Eikö >> lisää perään ja >ylikirjoita. Paitsi jos ympäristön asetukset blokkaa > toiminnan mikäli tiedosto on jo olemassa, jolloin joutuu käyttämään >|.

import_digiroad_shapefiles.sh line 27 at r1 (raw file):

Tsek. Ei tuosta haittaa ole, Näytti virheelä 🙂